Conversation
Ehyiah
force-pushed
the
trad-fr-hot-reload
branch
from
1410da5 to
588f952
Compare
There was a problem hiding this comment.
Pull request overview
Adds French documentation for FrankenPHP’s Hot Reload feature, aligning the docs/fr set with the existing English docs/hot-reload.md content.
Changes:
- Added a full French translation of the Hot Reload documentation page (
docs/fr/hot-reload.md). - Included configuration examples and client-side integration details (Mercure + idiomorph / frankenphp-hot-reload) in French.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Alexandre Daubois <2144837+alexandre-daubois@users.noreply.github.com>
| Bien que le serveur détecte les modifications, le navigateur doit s'abonner à ces événements pour mettre à jour la page. | ||
| FrankenPHP expose l'URL du Hub Mercure à utiliser pour s'abonner aux modifications de fichiers via la variable d'environnement `$_SERVER['FRANKENPHP_HOT_RELOAD']`. | ||
|
|
||
| Une bibliothèque JavaScript pratique, [frankenphp-hot-reload](https://www.npmjs.com/package/frankenphp-hot-reload), est également disponible pour gérer la logique côté client. |
There was a problem hiding this comment.
"Une bibliothèque JavaScript pratique" feels weird, like an automatic translation. I would remove "pratique" to sound more natural.
| FrankenPHP expose l'URL du Hub Mercure à utiliser pour s'abonner aux modifications de fichiers via la variable d'environnement `$_SERVER['FRANKENPHP_HOT_RELOAD']`. | ||
|
|
||
| Une bibliothèque JavaScript pratique, [frankenphp-hot-reload](https://www.npmjs.com/package/frankenphp-hot-reload), est également disponible pour gérer la logique côté client. | ||
| Pour l'utiliser, ajoutez ce qui suit à votre layout principal : |
There was a problem hiding this comment.
Same with "ce qui suit". We would better say "le code suivant" or something like that
| ### Mode Worker | ||
|
|
||
| Si vous exécutez votre application en [mode Worker](worker.md), le script de votre application reste en mémoire. | ||
| Cela signifie que les modifications de votre code PHP ne seront pas reflétées immédiatement, même si le navigateur se recharge. |
There was a problem hiding this comment.
The browser doesn't reload itself but reloads the page, right?
| Si vous exécutez votre application en [mode Worker](worker.md), le script de votre application reste en mémoire. | ||
| Cela signifie que les modifications de votre code PHP ne seront pas reflétées immédiatement, même si le navigateur se recharge. | ||
|
|
||
| Pour la meilleure expérience développeur, vous devriez combiner `hot_reload` avec [la sous-directive `watch` dans la directive `worker`](config.md#surveillance-des-modifications-de-fichier). |
There was a problem hiding this comment.
We don't use "vous devriez", but "il est possible de combiner" for example, which sounds less robotic as well
| } | ||
| ``` | ||
|
|
||
| ### Comment ça fonctionne |
There was a problem hiding this comment.
Here, it sounds a bit to oral, doesn't it? I would go for "Fonctionnement interne" or similar
|
|
||
| FrankenPHP inclut une fonctionnalité de **hot reload** intégrée, conçue pour améliorer considérablement l'expérience développeur. | ||
|
|
||
|  |
There was a problem hiding this comment.
Why do we use Mercure here?
3 participants
Hello, there doesn't seem to be an open MR for translating the new hot reload feature. Here's what I can offer.